-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump jupyter-server-proxy for security update. #5807
Conversation
There's a dependency problem in CI.
thanks @ryanlovett ! we should talk about testing/deployment before any merging takes place. :) |
@shaneknapp Sure, what did you have in mind? |
as discussed in slack, it's a proxy for other apps so i'd make sure everything works. plus it's touching (almost) every hub. links provided by @ryanlovett regarding the upstream changes: |
@shaneknapp +1. I've merged, and will post follow-up about testing hubs in this thread. |
Successfully tested
/rstudio, /desktop, and /vscode where available. Used |
@shaneknapp I think this is good to merge to prod whenever. |
GHSA-fvcq-4x64-hqxr